Skip to content

Conversation

Steffengreiner
Copy link
Contributor

PR Goal
This PR enables the deployment of thexml-manager-lib as an OSGI Bundle and in a non OSGI context by introducing two distinct profiles in the pom.xml.
Additionally it includes the jaxb-core dependency which was removed in JDK 11.

Additional Note
This PR adds the missing commits that were added directly to master to development to enable uniform development in the future.

wow-such-code and others added 18 commits July 24, 2018 15:28
* prepare next development version

* remove deprecated class

* new snapshot version: test of new model and parser

* more testing

* add hashcode and equals functions

* easier validation, proper logging

* check for empty factors, add xml validator, fix: factor values were not properly created

* add unit tests and schema resource

* unstable update

* renamed and added update functionality

* formatting

* add tests

* update methods, slight restructuring

* new constructor

* add test

* set deprecated

* updating factor levels will now remove identifiers from old factor levels

* add setter for value field

* add test

* add functionality to return all factos and properties for a single sample

* add functionality to keep technology types without attached sample ids
@codecov
Copy link

codecov bot commented Dec 3, 2020

Codecov Report

❗ No coverage uploaded for pull request base (development@852976a). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             development      #4   +/-   ##
=============================================
  Coverage               ?   0.00%           
=============================================
  Files                  ?      34           
  Lines                  ?    1075           
  Branches               ?     237           
=============================================
  Hits                   ?       0           
  Misses                 ?    1075           
  Partials               ?       0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 852976a...fda4cfd. Read the comment docs.

<groupId>life.qbic</groupId>
<artifactId>parent-pom</artifactId>
<version>2.4.0-SNAPSHOT</version>
<version>3.0.0</version>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we want to keep the parent pom? I thought we remove it for osgi projects 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good Catch! However I'd rather keep it in the current state for this PR and remove the Parent Pom in a seperate PR since this PR is pretty big already.

Copy link
Contributor

@jenniferboedker jenniferboedker Dec 3, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, I am fine with that. Good idea to make this in a separate PR. Do you want to write an issue or so, just that we won't forget?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is really important therefore I want to make sure this really will be tackled at some point

Copy link
Contributor

@jenniferboedker jenniferboedker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some minor comments. Furthermore, why is the license removed?

Other than that good work.

@Steffengreiner
Copy link
Contributor Author

The license was redrafted(deleted and added seperately) by SF and pushed directly to the master branch.
The license of the master branch is included in this branch and should not be changed.
The development branch of this lib was behind the master branch which resulted in a confusing git tracking.

Copy link
Contributor

@jenniferboedker jenniferboedker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lvgtm

@Steffengreiner Steffengreiner merged commit b76af2e into development Dec 3, 2020
@Steffengreiner Steffengreiner deleted the refactor/OSGI-RDY branch December 3, 2020 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants